Skip to content

DOCSP-46750 Kotlin Sync Title Change #66

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Feb 27, 2025

Conversation

shuangela
Copy link
Collaborator

@shuangela shuangela commented Feb 25, 2025

Pull Request Info

PR Reviewing Guidelines

JIRA - https://jira.mongodb.org/browse/DOCSP-46750

Staging Links

  • get-started
  • get-started/run-sample-query
  • Self-Review Checklist

    • Is this free of any warnings or errors in the RST?
    • Did you run a spell-check?
    • Did you run a grammar-check?
    • Are all the links working?
    • Are the facets and meta keywords accurate?

    Copy link

    netlify bot commented Feb 25, 2025

    Deploy Preview for docs-kotlin-sync ready!

    Name Link
    🔨 Latest commit 11f41c3
    🔍 Latest deploy log https://app.netlify.com/sites/docs-kotlin-sync/deploys/67c09b873831d500084d524c
    😎 Deploy Preview https://deploy-preview-66--docs-kotlin-sync.netlify.app
    📱 Preview on mobile
    Toggle QR Code...

    QR Code

    Use your smartphone camera to open QR code link.

    To edit notification comments on pull requests, go to your Netlify site configuration.

    Copy link
    Collaborator

    @mongoKart mongoKart left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    lgtm

    @shuangela shuangela requested a review from sarahemlin February 26, 2025 21:10
    Copy link
    Collaborator

    @sarahemlin sarahemlin left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    Ok, I see that these all had pretty much the same "bad" title and the same "good" title--same action for all, then! New URL + redirect.

    Copy link
    Collaborator

    @mongoKart mongoKart left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    LGTM w/ a fix and a question

    config/redirects Outdated
    @@ -8,3 +8,4 @@ symlink: current -> master

    raw: ${prefix}/ -> ${base}/current/
    raw: ${prefix}/master -> ${base}/upcoming/
    [v5.1-*]: ${prefix}/${version}/get-started/connect-to-mongodb -> ${base}/get-started/run-sample-query
    Copy link
    Collaborator

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    https://www.mongodb.com/docs/meta/redirects/#rules

    Suggested change
    [v5.1-*]: ${prefix}/${version}/get-started/connect-to-mongodb -> ${base}/get-started/run-sample-query
    [v5.1-*]: ${prefix}/${version}/get-started/connect-to-mongodb/ -> ${base}/get-started/run-sample-query/

    Copy link
    Collaborator Author

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    Ah thanks for the catch!

    config/redirects Outdated
    @@ -8,3 +8,4 @@ symlink: current -> master

    raw: ${prefix}/ -> ${base}/current/
    raw: ${prefix}/master -> ${base}/upcoming/
    [v5.1-*]: ${prefix}/${version}/get-started/connect-to-mongodb -> ${base}/get-started/run-sample-query
    Copy link
    Collaborator

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    q: just double-checking that this change should apply only to v5.1 and above?

    Copy link
    Collaborator Author

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    5.1 is the lowest version in the version flipper: https://www.mongodb.com/docs/languages/kotlin/kotlin-sync-driver/v5.1/! Should I just do raw instead if that is the case?

    Copy link
    Collaborator Author

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    Resolved in slack dms

    Copy link
    Collaborator

    @mongoKart mongoKart left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    lgtm

    @shuangela shuangela merged commit eee6b32 into mongodb:master Feb 27, 2025
    6 checks passed
    github-actions bot pushed a commit that referenced this pull request Feb 27, 2025
    * change titles
    
    * change url and add redirect
    
    * add slash
    
    * add back old ref based on rea's feedback
    
    (cherry picked from commit eee6b32)
    Copy link

    The backport to v5.1 failed:

    The process '/usr/bin/git' failed with exit code 1
    

    To backport manually, run these commands in your terminal:

    # Fetch latest updates from GitHub
    git fetch
    # Create a new working tree
    git worktree add .worktrees/backport-v5.1 v5.1
    # Navigate to the new working tree
    cd .worktrees/backport-v5.1
    # Create a new branch
    git switch --create backport-66-to-v5.1
    # Cherry-pick the merged commit of this pull request and resolve the conflicts
    git cherry-pick -x --mainline 1 eee6b3290896ed5340d89315e2efc8c872fea2a8
    # Push it to GitHub
    git push --set-upstream origin backport-66-to-v5.1
    # Go back to the original working tree
    cd ../..
    # Delete the working tree
    git worktree remove .worktrees/backport-v5.1

    Then, create a pull request where the base branch is v5.1 and the compare/head branch is backport-66-to-v5.1.

    github-actions bot pushed a commit that referenced this pull request Feb 27, 2025
    * change titles
    
    * change url and add redirect
    
    * add slash
    
    * add back old ref based on rea's feedback
    
    (cherry picked from commit eee6b32)
    shuangela added a commit that referenced this pull request Feb 27, 2025
    * change titles
    
    * change url and add redirect
    
    * add slash
    
    * add back old ref based on rea's feedback
    
    (cherry picked from commit eee6b32)
    
    Co-authored-by: shuangela <[email protected]>
    shuangela added a commit that referenced this pull request Feb 27, 2025
    * change titles
    
    * change url and add redirect
    
    * add slash
    
    * add back old ref based on rea's feedback
    
    (cherry picked from commit eee6b32)
    
    Co-authored-by: shuangela <[email protected]>
    shuangela added a commit to shuangela/docs-kotlin-sync that referenced this pull request Feb 27, 2025
    * change titles
    
    * change url and add redirect
    
    * add slash
    
    * add back old ref based on rea's feedback
    
    (cherry picked from commit eee6b32)
    shuangela added a commit to shuangela/docs-kotlin-sync that referenced this pull request Feb 27, 2025
    * change titles
    
    * change url and add redirect
    
    * add slash
    
    * add back old ref based on rea's feedback
    
    (cherry picked from commit eee6b32)
    shuangela added a commit that referenced this pull request Feb 27, 2025
    * change titles
    
    * change url and add redirect
    
    * add slash
    
    * add back old ref based on rea's feedback
    
    (cherry picked from commit eee6b32)
    shuangela added a commit that referenced this pull request Feb 27, 2025
    * DOCSP-46750 Kotlin Sync Title Change (#66)
    
    * change titles
    
    * change url and add redirect
    
    * add slash
    
    * add back old ref based on rea's feedback
    
    (cherry picked from commit eee6b32)
    
    * imports in vale action fix
    
    * Delete .github/workflows/check-autobuilder.yml
    
    * cherrypick
    
    ---------
    
    Co-authored-by: Rea Rustagi <[email protected]>
    shuangela added a commit that referenced this pull request Feb 27, 2025
    * DOCSP-46750 Kotlin Sync Title Change (#66)
    
    * change titles
    
    * change url and add redirect
    
    * add slash
    
    * add back old ref based on rea's feedback
    
    (cherry picked from commit eee6b32)
    
    * imports in vale action fix
    
    * Delete .github/workflows/check-autobuilder.yml
    
    * cherrypick
    
    * realized duplicate /docs in url redirect (#75)
    
    (cherry picked from commit 92549a3)
    
    cherrypick
    
    ---------
    
    Co-authored-by: Rea Rustagi <[email protected]>
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    3 participants